Skip to content

Conversation

@FGasper
Copy link
Contributor

@FGasper FGasper commented May 4, 2022

Issue #19690

@toddr toddr requested review from demerphq and leonerd May 4, 2022 16:39
Copy link
Contributor

@rwp0 rwp0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would be good to explicitly mention kernels/distros that do so in the comment as a short example

@FGasper
Copy link
Contributor Author

FGasper commented May 5, 2022

would be good to explicitly mention kernels/distros that do so in the comment as a short example

I don’t disagree, but I don’t know the full spate of systems that have been tested here. I’ll add a couple notes, though.

Copy link
Contributor

@rwp0 rwp0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's better, thanks

@toddr toddr self-requested a review May 11, 2022 15:19
Copy link
Contributor

@leonerd leonerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@leonerd
Copy link
Contributor

leonerd commented May 13, 2022

This has four approves now, and is a 5.36 release blocker. Can you merge it now please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants